Skip to content

Add proxy corrections and fix rsync permission problem on MacOS#24

Closed
MBARIMike wants to merge 13 commits intomainfrom
pr/17
Closed

Add proxy corrections and fix rsync permission problem on MacOS#24
MBARIMike wants to merge 13 commits intomainfrom
pr/17

Conversation

@MBARIMike
Copy link
Contributor

This should replace #17 as it includes all of those commits.

andermi and others added 13 commits April 14, 2025 19:14
Signed-off-by: Michael Anderson <anderson@mbari.org>
Signed-off-by: Michael Anderson <anderson@mbari.org>
Signed-off-by: Michael Anderson <anderson@mbari.org>
…le from resampled_nc

Signed-off-by: Michael Anderson <anderson@mbari.org>
Signed-off-by: Michael Anderson <anderson@mbari.org>
Signed-off-by: Michael Anderson <anderson@mbari.org>
Signed-off-by: Michael Anderson <anderson@mbari.org>
Signed-off-by: Michael Anderson <anderson@mbari.org>
…dinos_threshold

Signed-off-by: Michael Anderson <anderson@mbari.org>
MacOS 15.5 gave "fchmod: Permission denied" errors with rsync to smb mounted volumes. shutil.copyfile() replaces that.
@MBARIMike MBARIMike mentioned this pull request May 27, 2025
@MBARIMike
Copy link
Contributor Author

As a sanity check is are screen shots of before and after of biolume_proxy_diatoms at http://stoqs.mbari.org/p/3_RVFXo:

Before (from stoqs_all_dorado):
Screenshot 2025-05-27 at 1 06 38 PM

After (from stoqs_all_dorado_t):
Screenshot 2025-05-27 at 1 06 52 PM

@MBARIMike
Copy link
Contributor Author

The Parameter plots show less spikiness at the high values of biolume_proxy_diatoms for the corrected data:

Before (from stoqs_all_dorado):
Screenshot 2025-05-27 at 1 12 35 PM

After (from stoqs_all_dorado_t):
Screenshot 2025-05-27 at 1 12 50 PM

@MBARIMike MBARIMike mentioned this pull request May 28, 2025
@MBARIMike
Copy link
Contributor Author

Replaced by #25.

@MBARIMike MBARIMike closed this May 28, 2025
@MBARIMike MBARIMike deleted the pr/17 branch June 12, 2025 18:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants